Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EXIF Support #2206

Merged
merged 1 commit into from
Jul 13, 2015
Merged

EXIF Support #2206

merged 1 commit into from
Jul 13, 2015

Conversation

pjdufour
Copy link
Member

Duplicate of PR #2205 but squashed commits.

This PR adds an Exif contrib app. This app is disabled by default. If enabled by the EXIF_ENABLED settings flag, this app parses Exif tags from JPEGs uploaded as documents and automatically generates metadata, including date, keywords, and a brief abstract. It uses from PIL import Image, ExifTags, so doesn't require any changes to dependencies.

@ingenieroariel
Copy link
Member

I like this PR.

Why did Travis fail?

@pjdufour
Copy link
Member Author

pjdufour commented Jul 8, 2015

I'll try the tests manually sometime this week.

@pjdufour
Copy link
Member Author

pjdufour commented Jul 9, 2015

Added blank tests.py to see if that would resolve, but it didn't. The first error I get is the following. Sorry about a lot of text. Does anyone know off the top of their head what's going on? Otherwise, I think we'll have to postpone. Looks random, so probably hard to debug.

ERROR: testGeoTiffValidation (geonode.layers.tests.LayersTest)

Traceback (most recent call last):
File "/home/vagrant/geonode/geonode/layers/tests.py", line 60, in setUp
create_models(type='layer')
File "/home/vagrant/geonode/geonode/base/populate_test_data.py", line 205, in create_models
l.save()
File "/home/vagrant/.venvs/geonode/local/lib/python2.7/site-packages/polymorphic/polymorphic_model.py", line 90, in save
return super(PolymorphicModel, self).save(_args, *_kwargs)
File "/home/vagrant/.venvs/geonode/local/lib/python2.7/site-packages/django/db/models/base.py", line 545, in save
force_update=force_update, update_fields=update_fields)
File "/home/vagrant/.venvs/geonode/local/lib/python2.7/site-packages/django/db/models/base.py", line 582, in save_base
update_fields=update_fields, raw=raw, using=using)
File "/home/vagrant/.venvs/geonode/local/lib/python2.7/site-packages/django/dispatch/dispatcher.py", line 185, in send
response = receiver(signal=self, sender=sender, **named)
File "/home/vagrant/geonode/geonode/geoserver/signals.py", line 165, in geoserver_post_save
workspace=instance.workspace)
File "/home/vagrant/.venvs/geonode/local/lib/python2.7/site-packages/geoserver/catalog.py", line 601, in get_resource
store = self.get_store(store, workspace)
File "/home/vagrant/.venvs/geonode/local/lib/python2.7/site-packages/geoserver/catalog.py", line 259, in get_store
raise FailedRequestError("No store found named: " + name)
FailedRequestError: No store found named:

@pjdufour
Copy link
Member Author

pjdufour commented Jul 9, 2015

Looks like it ran fine on Travis CI. Weird?

@simod
Copy link
Member

simod commented Jul 9, 2015

@pjdufour this is the error that usually occurs when you run unit tests having geoserver running. Is that the case?

simod added a commit that referenced this pull request Jul 13, 2015
@simod simod merged commit c2c1a6f into GeoNode:master Jul 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants